Repository navigation
#83 #84 #85 Remove the broken UploadAndExecuteAction, LogRecord::hasContext(), ProcessHandler default level, declare symfony/ux-twig-component - #86
Merged
Conversation
…hasContext(), ProcessHandler default report level, declare symfony/ux-twig-component Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…cheduleCrudController Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
This was referenced Oct 2, 2026
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Fixes #83.
Fixes #84.
Fixes #85.
UploadAndExecuteAction(/process/upload-and-execute), andProcessUploadFileType/ProcessConfigurationValueResolver(and its service), only used by it. The action was unreachable from the UI, undocumented, and always failed (GET:No process with code :,POST:Unable to find template "@CleverAgeUiProcess/admin/process/upload_and_execute.html.twig"); uploading a file to run a process is handled byLaunchAction(entrypoint_type: file).LogRecord::hasContext(), used by the "Has context info ?" column ofLogRecordCrudController;contextIsEmpty()deprecated (unchanged behaviour: it returnstruewhen the context is not empty).ProcessHandler::$reportIncrementLeveldefaults toLevel::Warning, aslogs.report_increment_level;symfony/ux-twig-componentdeclared inrequire(^2.32 || ^3.0, as EasyAdmin), it is used by theBootstrapModalcomponent.Tests:
LogRecordTest(hasContext(), deprecatedcontextIsEmpty()),ProcessHandlerTest(default and configured report level, no write without filename). Without the fix, the two tests onhasContext()and the default level fail. The 2 PHPUnit deprecations come fromphpunit.xml.dist(executionOrder="depends", schema), as in the other bridges.Checked in process-bundle-demo with this branch installed (logged in): the container compiles, the
process_upload_and_executeroute is gone, and the process list, process launch (file upload form ofdemo.upload_and_run), executions, logs (with the "Has context info ?" column) and users pages answer200.Also documents, with a comment on
ProcessScheduleCrudController::schedulerIsRunning(), that the scheduler worker detection (ps -faux) only finds workers running on the same host as the web server (limitation already described in05-scheduler.md).Also fixes the v3.0.2 CHANGELOG entry (separate commit): broken GHSA link (
…/security/advisories/...) and hard line wrapping.Not in this PR: the full test coverage of #81.
Requirements
Breaking changes
process_upload_and_executeroute and theUploadAndExecuteAction,ProcessUploadFileTypeandProcessConfigurationValueResolverclasses are removed (the route always failed).LogRecord::contextIsEmpty()is deprecated.🤖 Generated with Claude Code